+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-42885/37023

• This PR adds an extra 36KB to repository

A new Pull Request was created by @theochatzis (Theo Chatzistavrou) for master.

It involves the following packages:

• HLTrigger/Configuration (hlt)

@missirol, @cmsbuild, @mmusich, @Martin-Grunewald can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @rovere, @SohamBhattacharya, @missirol, @silviodonato this is something you requested to watch as well.
@rappoccio, @sextonkennedy, @antoniovilela you are the release manager for this.

cms-bot commands are listed here

@cmsbuild, please test

@cmsbuild please abort

@theochatzis @cms-sw/alca-l2 I don't see the phase-2 version of the AK4PFPuppiHLT in the current phase2_realistic GlobalTag:

conddb list 132X_mcRun4_realistic_v1 | grep AK4PFPuppiHLT
[2023-09-27 14:33:01,837] INFO: Connecting to pro [frontier://PromptProd/cms_conditions]
JetCorrectionsRecord                                    AK4PFPuppiHLT                                                      JetCorrectorParametersCollection_Run3Summer21_MC_AK4PFPuppiHLT

please provide a new Global Tag.

assign alca

New categories assigned: alca

@saumyaphor4252,@perrotta,@consuegs you have been requested to review this Pull request/Issue and eventually sign? Thanks

@theochatzis Here's the GT 132X_mcRun4_realistic_v2

• Difference from the last GT: https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts/132X_mcRun4_realistic_v1/132X_mcRun4_realistic_v2
  • Only the new AK4PFPuppiHLT tag JetCorrectorParametersCollection_Phase2Spring23ticlv4_MC_AK4PFPuppiHLT_v1
• More details in this CMS Talk post https://cms-talk.web.cern.ch/t/mc-phase2hlt-new-jet-energy-corrections-for-ak4puppi-jets/29741

Great, thanks a lot Saumya! :) Ok added this tag in autoCond.

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-42885/37034

• This PR adds an extra 32KB to repository

@cmsbuild, please test

@cmsbuild ping?

@smuzaffar the bot looks stuck here, can you please have a look? thank you.

code-checks

I am looking in to it

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-42885/37035

• This PR adds an extra 16KB to repository

despite this

+code-checks

It still says

Pull request #42885 was updated. @perrotta, @consuegs, @Martin-Grunewald, @missirol, @saumyaphor4252, @mmusich can you please check and sign again.

@cmsbuild, please test

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3b6fa3/34955/summary.html
COMMIT: 8aa8cea
CMSSW: CMSSW_13_3_X_2023-09-28-1100/el8_amd64_gcc11
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/42885/34955/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• You potentially removed 1 lines from the logs
• Reco comparison results: 10 differences found in the comparisons
• DQMHistoTests: Total files compared: 50
• DQMHistoTests: Total histograms compared: 3358320
• DQMHistoTests: Total failures: 34
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3358264
• DQMHistoTests: Total skipped: 22
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 49 files compared)
• Checked 214 log files, 167 edm output root files, 50 DQM output files
• TriggerResults: found differences in 3 / 48 workflows

+hlt

• changes in HLT_*PuppiJet* paths in TriggerResults: look expected;
• some visible changes in e.g. in /HLT/HLTJETMET/HLT_AK4PFPuppiJet* in the DQM bin by bin comparisons

+alca

• differences in 24834.0, 24834.911, 24896.0, 24900.0, 25034.999 expected from the GT change
• Other diffs only in MsgLogger

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @antoniovilela, @sextonkennedy, @rappoccio (and backports should be raised in the release meeting by the corresponding L2)

+1